Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Sep 29, 2025

Summary

Fixed broken W3.org links in Table Cell Navigation documentation that were returning 404 errors. The W3C ARIA Authoring Practices Guide (APG) has restructured their URLs, and the old links were no longer valid.

Problem

The Table Cell Navigation example documentation contained links to https://www.w3.org/WAI/ARIA/apg/example-index/grid/dataGrids which now return 404 errors. This affected developers trying to access the official ARIA grid pattern documentation for implementing accessible table navigation.

Solution

Updated all references to use the new correct URL structure:

  • From: https://www.w3.org/WAI/ARIA/apg/example-index/grid/dataGrids
  • To: https://www.w3.org/WAI/ARIA/apg/patterns/grid/examples/data-grids/

Files Changed

  • packages/react-components/react-table/stories/src/Table/CellNavigation.stories.tsx - Updated Storybook documentation parameters
  • packages/react-components/react-table/stories/src/Table/TableDescription.md - Updated main Table component description
  • packages/react-components/react-table/library/docs/Spec.md - Updated both inline links and accessibility resources list

Impact

  • ✅ Developers can now access working links to W3C ARIA grid pattern examples
  • ✅ Table Cell Navigation documentation is fully functional
  • ✅ Maintains consistency across all Table-related documentation
  • ✅ No breaking changes to component functionality

Closes #[issue-number]

Original prompt

This section details on the original issue you should resolve

<issue_title>[Docs]: Table Cell Navigation example has broken link to w3.org</issue_title>
<issue_description>### Area

React Components (https://react.fluentui.dev)

What kind of documentation issue are you reporting?

  • Reporting a typo
  • Reporting a documentation bug
  • Documentation improvement
  • Documentation feedback

Is there a specific documentation page you are reporting?

https://react.fluentui.dev/?path=/docs/components-table--docs#cell-navigation

Description

The link aria-role="grid" pattern is broken. Maybe aria-role="grid" pattern is the new target page.

Validations

  • Check that there isn't already an issue that requests the same feature to avoid creating a duplicate.</issue_description>

<agent_instructions>use https://www.w3.org/WAI/ARIA/apg/patterns/grid/examples/data-grids/ to fix the link as the current points to 404</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes #35197

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot AI requested a review from Hotell September 29, 2025 15:40
Copilot finished work on behalf of Hotell September 29, 2025 15:40
@Hotell Hotell marked this pull request as ready for review October 1, 2025 09:06
@Hotell Hotell requested a review from a team as a code owner October 1, 2025 09:06
@Hotell Hotell enabled auto-merge (squash) October 1, 2025 09:06
@Hotell Hotell force-pushed the copilot/fix-9681b6cd-23aa-4c36-8cc4-fd859227fa72 branch from e828dd1 to ffb9238 Compare October 21, 2025 13:01
@github-actions
Copy link

📊 Bundle size report

✅ No changes found

@github-actions
Copy link

Pull request demo site: URL

This mode is the default, there is no keyboard navigation possible in the table content. However, this does not
include the header cells which can be sortable. They are covered above.

#### cell

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🕵🏾‍♀️ visual changes to review in the Visual Change Report

vr-tests-react-components/Charts-DonutChart 1 screenshots
Image Name Diff(in Pixels) Image Type
vr-tests-react-components/Charts-DonutChart.Dynamic.default.chromium.png 27053 Changed
vr-tests-react-components/Positioning 2 screenshots
Image Name Diff(in Pixels) Image Type
vr-tests-react-components/Positioning.Positioning end.updated 2 times.chromium.png 702 Changed
vr-tests-react-components/Positioning.Positioning end.chromium.png 278 Changed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Docs]: Table Cell Navigation example has broken link to w3.org

2 participants